Skip to content

Conversation

pYassine
Copy link
Contributor

No description provided.

@revu-bot revu-bot bot requested a review from revu-bot August 13, 2025 19:05
@pYassine pYassine force-pushed the feat-dsfr-mon-domifa branch from 6de7b54 to 7402835 Compare August 18, 2025 15:20
@pYassine pYassine temporarily deployed to build-review-auto August 18, 2025 15:20 — with GitHub Actions Inactive
@pYassine pYassine temporarily deployed to build-review-auto August 18, 2025 15:20 — with GitHub Actions Inactive
@pYassine pYassine temporarily deployed to build-review-auto August 18, 2025 15:20 — with GitHub Actions Inactive
@pYassine pYassine marked this pull request as ready for review August 18, 2025 17:01
Copy link
Collaborator

@revu-bot revu-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

An error occurred: ## ⚠️ PR Review Skipped

Issues Found

1. This PR changes 49 files, which exceeds the limit of 25 files.

Suggestion: Consider breaking this PR into smaller, more focused changes. Large PRs are harder to review effectively and may contain unrelated changes.

PR Metrics

  • Total files changed: 49
  • Reviewable files: 49
  • Diff size: 2190 lines
  • Documentation files: 0
  • Largest file change: 408 lines
  • Addition/Deletion ratio: 0.93

This validation helps ensure the bot focuses on PRs where automated review provides the most value. You can adjust these limits in your .revu.yml configuration file.

Revu logs

@pYassine pYassine changed the title feat(mon-domifa): init dsfr Admin & Mon DomiFa 1️⃣ feat(mon-domifa): init dsfr Admin & Mon DomiFa Sep 9, 2025
@hbalty hbalty force-pushed the feat-dsfr-mon-domifa branch from 7402835 to 1bd54cf Compare September 17, 2025 15:17
@hbalty hbalty temporarily deployed to build-review-auto September 17, 2025 15:17 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto September 17, 2025 15:17 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto September 17, 2025 15:17 — with GitHub Actions Inactive
Copy link

socket-security bot commented Sep 17, 2025

All alerts resolved. Learn more about Socket for GitHub.

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

View full report

@hbalty hbalty force-pushed the feat-dsfr-mon-domifa branch from 1bd54cf to bbb883c Compare September 17, 2025 16:15
@hbalty hbalty temporarily deployed to build-review-auto September 17, 2025 16:15 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto September 17, 2025 16:15 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto September 17, 2025 16:15 — with GitHub Actions Inactive
@hbalty hbalty force-pushed the feat-dsfr-mon-domifa branch from bbb883c to b45904d Compare September 18, 2025 14:43
@hbalty hbalty temporarily deployed to build-review-auto September 18, 2025 14:43 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto September 18, 2025 14:43 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto September 18, 2025 14:43 — with GitHub Actions Inactive
@hbalty hbalty force-pushed the feat-dsfr-mon-domifa branch from b45904d to d476827 Compare September 19, 2025 09:42
@hbalty hbalty temporarily deployed to build-review-auto September 19, 2025 09:42 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto September 19, 2025 09:42 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto September 19, 2025 09:42 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto September 19, 2025 09:42 — with GitHub Actions Inactive
* feat(mon-domifa): init dsfr

* fix(yarn): fix yarn.lock

* fix(yarn): fix yarn.lock

* feat(mon-domifa): init dsfr

* feat(front): footer dsfr portails admin & usagers

* fix(admin-usagers): mise en conformité à la NOUVELLE maquette

---------

Co-authored-by: Yassine R. <riffi.yassine@gmail.com>
* feat(usagers): integration header mon domifa

* feat(usagers): fix reviews

* fix(usagers): fix reviews
@hbalty hbalty temporarily deployed to build-review-auto September 23, 2025 21:31 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto September 23, 2025 21:31 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto September 23, 2025 21:31 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto September 23, 2025 21:31 — with GitHub Actions Inactive
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
19.4% Duplication on New Code (required ≤ 15%)

See analysis details on SonarQube Cloud

@tokenbureau
Copy link

tokenbureau bot commented Sep 23, 2025

🎉 Deployment for commit 0aa3986 :

Ingresses
Docker images
  • 📦 docker pull harbor.fabrique.social.gouv.fr/domifa/domifa/backend:sha-0aa3986ca98c52a6ba7f6705915c58930c12242d
  • 📦 docker pull harbor.fabrique.social.gouv.fr/domifa/domifa/frontend:sha-0aa3986ca98c52a6ba7f6705915c58930c12242d
  • 📦 docker pull harbor.fabrique.social.gouv.fr/domifa/domifa/portail-admins:sha-0aa3986ca98c52a6ba7f6705915c58930c12242d
  • 📦 docker pull harbor.fabrique.social.gouv.fr/domifa/domifa/portail-usagers:sha-0aa3986ca98c52a6ba7f6705915c58930c12242d
Debug

[severity]="toast.class === 'success' ? 'success' : 'error'"
[ariaRoleValue]="'alert'"
[hasAriaRole]="false"
[closeable]="toast.dissmissable"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ca ne fonctionne pas sur la démo. Je suis d'avis de le retirer pour l'instant, on va perdre du temps à essayer de corriger le souci

link: '/plan-site',
},
{
label: 'Nos partenaires',
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hbalty a retirer

link: '/cgu',
},
{
label: 'Polotiques de confidentialité',
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hbalty typo

données, consultez notre</span
>
<a
rel="noopener external"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

le "rel" est à retirer, on est sur un lien interne @hbalty

link: '/plan-site',
},
{
label: 'Nos partenaires',
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A retirer dans l'admin & mon domifa

@pYassine pYassine changed the title 1️⃣ feat(mon-domifa): init dsfr Admin & Mon DomiFa 🎨 DSFR - Admin & Mon DomiFa - Branche principale ⭐ Sep 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants